Skip to content

Conversation

jwwojak
Copy link
Contributor

@jwwojak jwwojak commented Sep 19, 2025

Overview

Revise and migrate OT-2 instruction manual into the new documentation system. Flex instruction manual is the model.

  • Revise: bring prose and contents up to current writing standards.
  • Remove: unnecessary contents, sections, and even chapters.
  • Migrate: get this stuff into MkDocs

Other changes: There should be no really new information added, unless something important is missing. This is old wine, a new bottle, and remove the sediment.

We will create separate branches from this for each chapter and merge those back into this branch after review.

Sandbox: https://sandbox.docs.opentrons.com/docs-ot2-manual-revisions/

References:

Test Plan and Hands on Testing

I can haz wordz. Plz read.
gudRigting

Changelog

2 new files for review:

  • introduction.md
  • regulatory.md

Review requests

If some text is familiar or seems familiar, that's because it may be. Some OT-2 sections are similar to the Flex manual. In many cases, this project will replace that older content with text from the Flex manual, and just change "Flex" to "OT-2."

Risk assessment

Low because this is documentation; medium because it is content an installed user-base relies on. Need to keep the revised manual useful for those folks.

Delete if the project shouldn't go forward.
Created sections to see what this could look like.

- placeholder section headers
- placeholder text
@jwwojak jwwojak self-assigned this Sep 19, 2025
Copy link

codecov bot commented Sep 22, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 24.67%. Comparing base (d958b55) to head (37da69e).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             edge   #19620      +/-   ##
==========================================
+ Coverage   23.04%   24.67%   +1.62%     
==========================================
  Files        3437     3445       +8     
  Lines      302623   304722    +2099     
  Branches    39867    40009     +142     
==========================================
+ Hits        69746    75188    +5442     
+ Misses     232854   229509    -3345     
- Partials       23       25       +2     
Flag Coverage Δ
app 3.14% <ø> (+2.82%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 245 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Moving on to other chapters
@jwwojak jwwojak marked this pull request as ready for review September 26, 2025 18:18
@ecormany ecormany added the DO NOT MERGE Indicates a PR should not be merged, even if there's a shiny green merge button available label Sep 29, 2025
Copy link
Collaborator

@emilyburghardt emilyburghardt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking pretty good so far, Joe! Left a few comments that might be in sections you haven't gotten to yet (apologies if so). Re-request review whenever more files are ready/the whole thing is ready and I'll look again.

I will say the sandbox isn't working for me (pulls up the docs site, but it doesn't include the OT-2 manual). Not sure if this is a known issue.

@@ -0,0 +1,10 @@
---
title: "Opentrons OT-2: Appendicies"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

appendices?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That shouldn't be in this branch. Removing.

@@ -0,0 +1,14 @@
---
title: "Opentrons OT-2 Liquid Handler Instruction Manual"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we just keep it "Opentrons OT-2 Instruction Manual"?

the Flex manual title doesn't include "liquid handler"


### Requirements

Warning: The Opentrons Liquid Handler can emit vibration while in operation. For
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

replace "Opentrons Liquid Hander" with OT-2 throughout section?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file shouldn't be in here. Just look at introduction.md and regulatory.md

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DO NOT MERGE Indicates a PR should not be merged, even if there's a shiny green merge button available docs mkdocs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants